Update to Pico-SDK 2.2.0 release, remove multi-lib LWIP, clean up CMake, clean up P.IO build hardcoded defines #3049
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Doesn't seem to have nay additions since the develop branch we were using, but this one is nice and official and tagged.
We've collected multiple -Wno-xxx settings while working around small issues with the SDK. Remove them and see where we stand
earlephilhower
force-pushed
the
yetanothersdkupdate
branch
from
3898b96 to
94272f7
Compare
We needed about 6 copies of the LWIP/BT library for each CPU when the lwip library was built as part of the SDK libpico. Remove LWIP from the SDK builds and replace it with the actual LWIP source (linked via `#include`) so we can adjust for IPv4/IPv^/32K/16K at runtime. Shrinks download size of a release significantly. Still need separate LWIP-BT library because the CYW43 driver is different when BT is enabled. Update and clean up the libpico CMakefile and common platform_def.txt Link to config includes, don't copy, where possible
earlephilhower
changed the title
earlephilhower
force-pushed
the
yetanothersdkupdate
branch
from
7dae7fa to
22ad092
Compare
earlephilhower
changed the title
earlephilhower
force-pushed
the
yetanothersdkupdate
branch
from
5ffc83d to
47079e6
Compare
|
@maxgerhardt if you could look over the changes I've done in your The defines set in the P.IO builds never really matched the ones in the IDE because they relied on cut-n-pasting (and editing for proper formatting) updates every time. Now, the Arduino IDE and the P.IO should pull chip-level defines from a GCC |
Accidentally defined CYW43_SUPPORTED for all boards, not just the ones with a CYW43 chip onboard.
Use newer script which parses local tzdata files
|
Changes to the |
No actual changes to output boot2 files, but if they ever do update we'll catch it.
earlephilhower
force-pushed
the
yetanothersdkupdate
branch
from
f23b2ca to
207f8f5
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Doesn't seem to have many additions since the develop branch we were using, but this one is nice and official and tagged.